-
Notifications
You must be signed in to change notification settings - Fork 223
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add support for Ollama, Palm, Claude-2, Cohere, Replicate, Llama2 CodeLlama (100+LLMs) [LiteLLM] #160
base: main
Are you sure you want to change the base?
Conversation
@loubnabnl @Muennighoff @infinitylogesh I believe LiteLLM makes it easier to benchmark LLMs - would love your feedback if not |
Thanks for the PR, what do you think @Muennighoff ? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Okay with me if we add those two comments so people know that it's the same as via OAI
Co-authored-by: Niklas Muennighoff <n.muennighoff@gmail.com>
Co-authored-by: Niklas Muennighoff <n.muennighoff@gmail.com>
@Muennighoff @loubnabnl updated based on feedback - thanks |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the integration! I left a small comment about the installation
@@ -12,6 +12,7 @@ | |||
"matplotlib==3.5.2", | |||
"numpy==1.21.6", | |||
"openai==0.23.0", | |||
"litellm==0.13.2", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That's not the right place for the requirement as this is for DS1000 task that doesn't support external APIs, it seems we didn't have openai
package properly added.
I would leave this list as it is and create a new variable
external_api_requirements=["litellm==0.13.2"]
then add it to the extras_require
dict here so people that want this evaluation install the package separately
This PR adds support for the above mentioned LLMs using LiteLLM https://github.com/BerriAI/litellm/
LiteLLM is a lightweight package to simplify LLM API calls - use any llm as a drop in replacement for gpt-3.5-turbo.
Example